Hard cut over Architecture renderer to Scryer view model#6504
Open
Nikolatesla-lj wants to merge 45 commits into
Open
Hard cut over Architecture renderer to Scryer view model#6504Nikolatesla-lj wants to merge 45 commits into
Nikolatesla-lj wants to merge 45 commits into
Conversation
…tegration # Conflicts: # package.json
Route default Architecture IPC, renderer session reads, and strict Scryer MCP writes through the Native Scryer Engine operation surface. Add edit-session lease/completion gate support plus catalog-backed node/link/group/source/drift coverage for Scryer 0.3 models.
Stabilize visible Architecture tab, canvas, group, source-link, and sync controls. Persist architecture tab chrome in workspace session patches and expand Electron e2e coverage to assert real UI interactions and .scryer/planned.scry effects.
# Conflicts: # package.json # pnpm-lock.yaml # src/main/persistence.ts # src/renderer/src/components/automations/automation-run-view-state.test.ts # src/renderer/src/components/automations/automation-usage-model.test.ts # src/renderer/src/components/tab-group/useTabGroupWorkspaceModel.ts # src/renderer/src/hooks/resolve-zoom-target.ts # src/renderer/src/lib/automation-session-reuse.test.ts # src/shared/automations-types.ts
Contributor
Author
|
Local release verification for c6e64d6:\n\n- PR branch is mergeable after merging origin/main at c0f4bdd.\n- corepack pnpm run tc: passed.\n- Targeted vitest suite: 11 files / 199 tests passed.\n- Electron Architecture e2e: tests/e2e/architecture-tab.spec.ts, 12 passed in 4.9m.\n- Docs status checked: #28/#29 and operation migration are already documented as complete; no docs diff was needed in this publish pass.\n\nGitHub currently reports no checks for the fork branch, so the above is the local verification record for this tip. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.scryerfile effects.Commits
59b4faaa5Hard cut over Architecture renderer to Scryer view modela86476f52Expand Architecture live UI coverage0581889d2Update Scryer migration status docs7288460b4Fix Architecture IPC test mock typing98cf51a7fReject legacy C4 raw writes on default Architecture modelVerification
corepack pnpm run tccorepack pnpm exec vitest run --config config/vitest.config.ts src/main/ipc/architecture.test.ts src/main/ipc/architecture-view-ipc.test.ts src/renderer/src/components/architecture/useArchitectureModelController.test.ts src/renderer/src/components/architecture/architecture-view-model.test.ts src/renderer/src/components/architecture/architecture-diagram-model.test.tscorepack pnpm exec electron-vite build --mode e2eSKIP_BUILD=1 corepack pnpm exec playwright test tests/e2e/architecture-tab.spec.ts --config tests/playwright.config.ts --project electron-headless --workers=1git diff --check dc1c488b1...HEADKnown Follow-up
GroupsViewstill uses catalogedscryer.group.setfor some whole-group updates such as drag/drop nesting and empty group flows. This is not a legacy C4 bypass, but it is not yet fully split into typedgroup.add/update/delete/moveoperations. Recommend trackinggroup.moveand typed group edit coverage as a follow-up ticket unless reviewer marks it blocking.